Skip to content

fix(pack): keep related memories self-contained#1110

Merged
kunickiaj merged 1 commit into
mainfrom
05-20-fix_pack_keep_related_memories_self-contained
May 21, 2026
Merged

fix(pack): keep related memories self-contained#1110
kunickiaj merged 1 commit into
mainfrom
05-20-fix_pack_keep_related_memories_self-contained

Conversation

@kunickiaj
Copy link
Copy Markdown
Owner

@kunickiaj kunickiaj commented May 21, 2026

Description

Fixes a pack compression regression where near-related memories were treated like duplicates and removed from default rendered context. Default packs now remain self-contained; near-related compression is limited to compact mode by default, with explicit ids mode available for tool-enabled benchmarks/agents.

Adds --compression-mode off|compact|ids, CODEMEM_PACK_COMPRESSION, and MCP compression_mode, plus clearer compact-mode fetch guidance for memory_get_observations.

Type of Change

  • 🚀 Feature (new functionality)
  • 🐛 Bug fix (fixes an issue)
  • 📚 Documentation (docs-only change)
  • 🔧 Maintenance (refactor, chore, CI, etc.)
  • 🧪 Testing (test-only changes)

Testing

  • Relevant checks pass locally (pnpm run tsc, pnpm run lint, pnpm run test)
  • Added/updated tests for changes
  • Manually verified changes work as expected

Targeted checks run:

  • pnpm exec vitest run packages/core/src/pack.test.ts packages/cli/src/commands/pack.test.ts
  • pnpm run tsc
  • pnpm run lint

Checklist

  • Code follows project style (pnpm run lint passes for touched files)
  • Self-review completed
  • Documentation updated (if needed)
  • No new warnings introduced

Copy link
Copy Markdown
Owner Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@kunickiaj kunickiaj marked this pull request as ready for review May 21, 2026 05:44
Copy link
Copy Markdown
Owner Author

kunickiaj commented May 21, 2026

Merge activity

  • May 21, 5:53 AM UTC: A user started a stack merge that includes this pull request via Graphite.
  • May 21, 5:53 AM UTC: @kunickiaj merged this pull request with Graphite.

@kunickiaj kunickiaj merged commit d54cdf9 into main May 21, 2026
11 checks passed
@kunickiaj kunickiaj deleted the 05-20-fix_pack_keep_related_memories_self-contained branch May 21, 2026 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant